Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/59670 policy time function extension #75

Closed

Conversation

Julian-Marco
Copy link

Dies ist ein erstes Beispiel für ein Policy Enforcement - Modul des EDC. Konkret erhält der Benutzer hiermit die Möglichkeit, konkrete Zeitpunkte oder Zeitfenster anzugeben, innerhalb derer ein Asset gesehen / verhandelt werden darf. Erstelle ich beispielsweise ein Asset mit einer Access Policy (Access Policies steuern im Allgemeinen die Sichtbarkeit vom Assets im Datenraum), die besagt, dass das Asset bis zum einschließlich 30.06. gesehen werden darf, so verhindert mein eigener EDC ab dem 01.07. automatisch, dass andere Teilnehmer das Asset in meinem internen Katalog wiederfinden können.

Allgemeiner liefert der PR das allgemeine Setup für das Hinzufügen weiterer eigener Extensions, wie zum Beispiel für die zu modifizierenden S3-Extensions.

@Julian-Marco Julian-Marco requested a review from T-Thurow June 28, 2023 06:09
@T-Thurow
Copy link

Ich würde die extensions, auch wenn diese von uns erstellt und gepflegt werden, nicht mit in dieses repository kopieren sondern an dieser Stelle mit git submodulen arbeiten. Der Code hier scheint ja vom Fraunhofer Institut zu sein und wird dementsprechend auch von denen gepflegt daher würde es sich anbieten das Repository als git submodule auszulegen und das bei update der EDC version mit zu updaten. Dann haben wir auch keinen fremden Code mit im Repository an dem wir garnicht mit gearbeitet haben.

@Julian-Marco
Copy link
Author

Der Code ist eigentlich eine Mischung aus diesem (nicht gemergten, da sich keiner drum kümmert) PR eclipse-edc/Samples#35 + https://github.com/FraunhoferISST/edc-samples/tree/feat/basic-policy-sample und von etwas das sovity gemacht hat: https://github.com/sovity/edc-extensions/blob/main/extensions/policy-time-interval/src/main/java/de/sovity/edc/extension/policy/TimeIntervalFunction.java Also die Idee ist beiden Fällen dieselbe, aber sovity hatte mit mehr Operatoren gearbeitet, weswegen ich den Teil einfach rauskopiert habe. Das Lizenz-Blabla ist da eigentlich nur ein Überbleibsel.

Also das ist in beiden Fällen nix was jetzt irgendwo als Extension im offiziellen EDC-Repo liegt. Sonst hätte man ja wie bei den anderen Extensions einfach die entsprechenden Maven artifacts verwenden können.

@Julian-Marco
Copy link
Author

Aber ja, ich sehe das Argument, dass das Potential für zusätzliche Maintenance liefert, ohne dass wir die Funktionalität jetzt "akut" brauchen. Würde es halt als Minimalbeispiel für Policy Enforcement betrachten, eher im didaktischen Sinne, damit man das mal gesehen hat.

@Julian-Marco Julian-Marco deleted the feature/59670_policy_time_function_extension branch July 18, 2023 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants